-
Notifications
You must be signed in to change notification settings - Fork 561
[release-4.19] CORENET-6148: Promote RouteAdvertisements feature gate to default feature set #2425
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: release-4.19
Are you sure you want to change the base?
Conversation
Signed-off-by: Arnab Ghosh <[email protected]>
@arghosh93: This pull request references CORENET-6148 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
Hello @arghosh93! Some important instructions when contributing to openshift/api: |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: arghosh93 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@arghosh93 Please link the associated SBAR in the PR description Also, schema checker is reporting existing issues that cannot be fixed, we can override that /override ci/prow/verify-crd-schema |
@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/verify-crd-schema In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
yea will open a SBAR and link it here. |
@arghosh93: This pull request references CORENET-6148 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
/hold |
/test verify-feature-promotion |
4 similar comments
/test verify-feature-promotion |
/test verify-feature-promotion |
/test verify-feature-promotion |
/test verify-feature-promotion |
@JoelSpeed I think these numbers do not add up
They run daily and they have been running for more than 14 days. What are we missing? |
If for any reason the cluster failed to come up one or two of those days, you'll have missed out on the data as the result wouldn't be recorded for a cluster bootstrapping fail. The best thing to do here would be to work out which job you need to run to trigger those tests and use gangway-cli to trigger say 5 more runs |
It's strange though that it just happens to those particular test cases if entire jobs failed to run. Will run gangway as you suggested, thanks. |
/test verify-feature-promotion |
@jcaamano we are almost there. |
/test verify-feature-promotion |
1 similar comment
/test verify-feature-promotion |
@arghosh93: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
The verify lane is looking much better. The tests are at the correct pass rate above 95% on the dualstack lanes we have.
It's failing for platform reasons. Is this something you can override @JoelSpeed or do we need to do anything on our side. As for the SBAR: I had opened it 20 days ago: https://docs.google.com/presentation/d/1Ak0bkXFkvoY8JxipMQGct8uCWroPNc84V64PjmXFyek/edit?slide=id.g9814768380_0_0#slide=id.g9814768380_0_0 haven't heard back though @davegord is helping me get some eyes on this. |
/hold please don't remove hold (@jcaamano will do it once we are ready on ovnk/cno side fully) we are only trying to ensure we get this PR to a ready state in parallel. |
This PR lifts the FG for RouteAdvertisements feature.
RouteAdvertisements feature gate was lifted in 4.20 via below PR:
#2225